Skip to content

chore: Update TESTED_FEATURES.md #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Max07B2 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore: Update TESTED_FEATURES.md #33

Max07B2 wants to merge 1 commit into from

Conversation

@Max07B2
Copy link
Contributor

@Max07B2 Max07B2 commented Feb 8, 2026 •
edited by coderabbitai bot
Loading

Tested the remaining todo-items for HW-V1 and updated the test matrix according to the following test scenario

Environment

  • HWv1
  • OSv2 (OpenMowerOS_20260203.img)
  • open_mower_ros: latest (v1.1.0)
  • Yard Force Classic 500 with mainboard 0_11_X_WT901
  • no NTRIP usage

Test scenarios

Test 1: only image defaults

  • Hostname (default) was openmower => ok
  • Default user/password was openmower/openmower => ok
  • Comitup AP appeared and was able to configure my WLAN => ok
  • I think internal LAN does not apply for HWv1 so I set "--"
  • connected a LAN cable to the RasPi and got an IP-address, browser connection with it works => ok
  • I have no experience with OpenOCD debugging but I did the following
    • openmower openocd
    • Telnet Connection on Port 4444: Prompt Open On-Chip Debugger
    • targets: 
      TargetName         Type       Endian TapName            State
--  ------------------ ---------- ------ ------------------ ------------
 0* rp2040.core0       cortex_m   little rp2040.cpu         halted
 1  rp2040.core1       cortex_m   little rp2040.cpu         halted
    • reset halt: 
      [rp2040.core0] halted due to debug-request, current mode: Thread
xPSR: 0xf1000000 pc: 0x000000ea msp: 0x20041f00
[rp2040.core1] halted due to debug-request, current mode: Thread
xPSR: 0xf1000000 pc: 0x000000ea msp: 0x20041f00
    • resume yields the start audio message
      I guess openOCD debugging works as well => ok

Test 2: set ssh key in imager

  • SSH session without password promt (for user openmower) works => ok
  • I also switched off the wifi to see if the Comitup AP re-appears - and it did as expected => ok

Summary by CodeRabbit

  • Documentation
    • Updated testing status matrix to reflect expanded feature coverage across hardware variants, including hostname defaults, authentication options, SSH public key support, and remote debugging capabilities.

@Max07B2
chore: Update TESTED_FEATURES.md
fbf21a7 
Tested the remaining todo-Items for HW-V1, details see pull request
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 8, 2026 •
edited
Loading

Walkthrough

A feature-test matrix in TESTED_FEATURES.md was updated to reflect completed features and hardware support. Multiple status cells were changed from incomplete (🟡) to complete (✅), including hostname defaults, SSH keys, and OpenOCD remote debugging. Minor content adjustments and formatting refinements were also applied.

Changes

Cohort / File(s) Summary
Feature Test Matrix Updates
TESTED_FEATURES.md		 Updated status indicators across multiple features (Hostname default, Default user/password, SSH public key, OpenOCD remote debugging) from incomplete (🟡) to complete (✅) across various hardware variants. Adjusted content for Internal LAN, No known Wi-Fi, Comitup captive portal, and Home LAN rows. Reformatted footnote reference and removed trailing newline.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • ClemensElflein/OpenMowerOS#29: Edits the same TESTED_FEATURES.md file with overlapping feature-test matrix status updates (Hostname entries, OpenOCD, and OpenMowerOS statuses).

Poem

🐰 The matrix blooms with checkmarks bright,
Status updates, row by row so right,
From yellow flags to verdant green,
Features tested, progress seen!
A doc refreshed, completeness near,
Our feature map grows crystal clear! ✅

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: Update TESTED_FEATURES.md' is directly related to the main change—it accurately describes that the TESTED_FEATURES.md file was updated with test results.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
TESTED_FEATURES.md (1)

34-34: Inconsistent Wi‑Fi styling.

The footnote uses "WiFi" while the rest of the document uses "Wi‑Fi" (with en-dash). Please standardize to "Wi‑Fi" for consistency throughout the document.

📝 Proposed fix for consistency 
-[^1]: You probably need to power-cycle your mower after entering your home WiFi credentials
+[^1]: You probably need to power-cycle your mower after entering your home Wi‑Fi credentials

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Max07B2